-
Notifications
You must be signed in to change notification settings - Fork 112
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NIDAQ: Added analog input functionality #514
base: main
Are you sure you want to change the base?
Conversation
ae12591
to
4afff6e
Compare
First version to see if the inputs are correctly detected and added to the hub
Basic implementation reading at ~10Hz
Added property handlers for the ExpectedMin- and ExpectedMaxVolts properties
4afff6e
to
41cc80b
Compare
Your branch compiles fine with the 9.2 version of NIDAQms that we are building against. There are a couple of compiler warnings, but those will be easy to fix before merging. Let me know if it is ready for merging. |
I just went through the compiler warnings and the ones that are left fall into two categories:
The NI DAQ card arrived last week, so I expect the other two tasks to be checked off somewhere this week |
I ran some test traces and measurements with our PCIe 6323 card and with the fixes the traces and continuous sampling are working as intended. Let me know if there are any more changes needed before merging. |
Expanded the driver to be able to read analog inputs and capture traces. Has been tested with NI MAX emulation software but not with actual hardware.
Things to check before merging: